app/src/main/java/com/texthip/thip/ui/feed/viewmodel/FeedViewModel.kt (1)

545-551: TODO 주석 확인 - API 연결 필요

현재 loadRecommendedFeeds() 함수가 빈 리스트를 반환하는 플레이스홀더 구현입니다. PR 설명에 따르면 API가 완성되면 서비스와 레포지토리만 구현하면 된다고 하셨는데, API 연결 일정이 확정되어 있나요? 추후 누락되지 않도록 이슈 트래킹을 권장드립니다.

app/src/main/java/com/texthip/thip/ui/feed/screen/FeedScreen.kt (1)

561-579: 추천 캐러셀 통합이 잘 구현되었습니다!

10번째 피드 항목 후에 추천 섹션을 삽입하는 로직이 올바르게 구현되었습니다. 여백과 구분선 처리가 적절하며, 빈 리스트 체크도 포함되어 있습니다.

다만, else 블록의 로직이 약간 복잡할 수 있습니다. 현재는 마지막 항목이 아닐 때만 여백과 구분선을 추가하는데, 추천 섹션이 있는 경우와 없는 경우를 구분하여 가독성을 높일 수 있습니다.

app/src/main/java/com/texthip/thip/ui/feed/component/RecommendFeedCard.kt (1)

56-63: 텍스트 처리 로직 개선 제안

현재 processedText는 줄바꿈 문자(\n)로 분할하여 3줄만 표시하지만, 이후 onTextLayout에서 실제 시각적 오버플로우를 감지합니다. 이 두 로직이 중복될 수 있습니다.

SavedFeedCard처럼 maxLines 속성만으로 텍스트를 제한하고 onTextLayout으로만 잘림을 감지하는 것이 더 간단하고 일관성 있을 수 있습니다.

예시:

-        // 추천글은 항상 3줄로 고정
-        val processedText = remember(feedItem.contentBody) {
-            val lines = feedItem.contentBody.split("\n")
-            if (lines.size <= maxTextLines) {
-                feedItem.contentBody
-            } else {
-                lines.take(maxTextLines).joinToString("\n")
-            }
-        }
-
         Column(
             modifier = Modifier
                 .fillMaxWidth()
                 .padding(12.dp)
         ) {
             // ... ProfileBar ...
             
             // ... ActionBookButton ...

             Column(
                 modifier = Modifier.clickable { onClick() },
                 verticalArrangement = Arrangement.Center,
                 horizontalAlignment = Alignment.CenterHorizontally
             ) {
                 Box {
                     Text(
-                        text = processedText,
+                        text = feedItem.contentBody,
                         style = typography.feedcopy_r400_s14_h20,
                         color = colors.White,
                         maxLines = maxTextLines,
                         modifier = Modifier.fillMaxWidth(),
                         onTextLayout = { textLayoutResult ->
                             isTextTruncated = textLayoutResult.hasVisualOverflow
                         }
                     )

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

Learnt from: rbqks529
PR: THIP-TextHip/THIP-Android#30
File: app/src/main/java/com/texthip/thip/ui/common/cards/CardItemRoomSmall.kt:108-112
Timestamp: 2025-07-01T07:19:10.981Z
Learning: The user rbqks529 prefers to extract hardcoded UI strings to StringResource files (strings.xml) rather than keeping them inline in Compose components, which is a good practice for maintainability and localization in Android development.